src/renderer/src/providers/CosmeticsProvider.jsx (3)

15-22: Gate verbose logs behind a debug flag and use console.debug

Avoid always-on logging in the renderer; gate it with RENDERER_VITE_* and demote to debug.

Apply this diff:

-    console.log(
+    if (import.meta.env.DEV || import.meta.env.RENDERER_VITE_DEBUG_STV === 'true') console.debug(
       `[CosmeticsStore] Upserting style for ${transformedUsername}`,
       {
         badgeId: userStyle?.style?.badge_id,
         paintId: userStyle?.style?.paint_id,
         entitlementId: body?.object?.id,
       },
     );

---

66-72: Same: gate catalog update logs

Keep cosmetics catalog logs behind a flag to reduce noise in production.

-    console.log(
+    if (import.meta.env.DEV || import.meta.env.RENDERER_VITE_DEBUG_STV === 'true') console.debug(
       `[CosmeticsStore] Updating global cosmetics catalog`,
       {
         badges: body?.badges?.length,
         paints: body?.paints?.length,
       },
     );

---

12-13: Normalize usernames consistently (hyphen vs underscore)

You store with toLowerCase() only, but producer code (ChatProvider) replaces “-” → “_”. This can cause cache misses on getUserStyle.

-    const transformedUsername = username.toLowerCase();
+    const transformedUsername = username.replaceAll("-", "_").toLowerCase();

-    const transformedUsername = username.toLowerCase();
+    const transformedUsername = username.replaceAll("-", "_").toLowerCase();

Also applies to: 49-50

src/renderer/src/providers/ChatProvider.jsx (4)

1896-1904: Make 7TV “received event” logs debug‑gated

This fires for every WS event; keep it dev/debug only.

-    console.log(
+    if (import.meta.env.DEV || import.meta.env.RENDERER_VITE_DEBUG_STV === 'true') console.debug(
       `[ChatProvider] Received 7TV event ${type} for ${chatroomId ?? 'broadcast'}`,
       {
         badgeCount: body?.badges?.length,
         paintCount: body?.paints?.length,
         entitlementUser: body?.object?.user?.username,
       },
     );

---

1912-1918: Gate cosmetic catalog logs

Reduce renderer noise; keep under RENDERER_VITE_DEBUG_STV.

-        console.log(
+        if (import.meta.env.DEV || import.meta.env.RENDERER_VITE_DEBUG_STV === 'true') console.debug(
           `[ChatProvider] Applying cosmetic catalog update for ${chatroomId ?? 'all chatrooms'}`,
           {
             badges: body?.badges?.length,
             paints: body?.paints?.length,
           },
         );

---

1922-1924: Avoid duplicating username canonicalization here

CosmeticsProvider should own normalization; pass the raw Kick username and let the store normalize consistently.

-        const transformedUsername = username?.replaceAll("-", "_").toLowerCase();
+        const transformedUsername = username; // normalization handled inside CosmeticsProvider

---

1924-1931: Gate entitlement processing logs

Same rationale: keep these as debug.

-        console.log(
+        if (import.meta.env.DEV || import.meta.env.RENDERER_VITE_DEBUG_STV === 'true') console.debug(
           `[ChatProvider] Processing entitlement for ${transformedUsername || 'unknown user'}`,
           {
             badgeId: body?.object?.user?.style?.badge_id,
             paintId: body?.object?.user?.style?.paint_id,
             chatroomId,
           },
         );

utils/services/seventv/sharedStvWebSocket.js (1)

177-179: Gate shared 7TV logs behind a single DEBUG flag; use console.debug

These logs are frequent. Add a module‑level DEBUG flag (VITE_* per guidelines for shared code) and switch to console.debug to avoid production noise.

-    console.log(
+    if (import.meta.env.VITE_DEBUG_7TV === 'true' || (typeof process !== 'undefined' && process.env.NODE_ENV === 'development')) console.debug(
       `[Shared7TV]: Registered chatroom ${chatroomId} (kick=${channelKickID}, stvUser=${stvId}, stvSet=${stvEmoteSetId})`,
     );

-    console.log(
+    if (import.meta.env.VITE_DEBUG_7TV === 'true' || (typeof process !== 'undefined' && process.env.NODE_ENV === 'development')) console.debug(
       `[Shared7TV]: Preparing subscriptions for chatroom ${chatroomId} (kick=${chatroomData.channelKickID}, emoteSet=${chatroomData.stvEmoteSetId})`,
     );

-      console.log(
+      if (import.meta.env.VITE_DEBUG_7TV === 'true' || (typeof process !== 'undefined' && process.env.NODE_ENV === 'development')) console.debug(
         `[Shared7TV]: Cosmetic subscription already active for Kick channel ${channelKickID} (chatroom ${chatroomId})`,
       );

-      console.log(
+      if (import.meta.env.VITE_DEBUG_7TV === 'true' || (typeof process !== 'undefined' && process.env.NODE_ENV === 'development')) console.debug(
         `[Shared7TV]: Entitlement subscription already active for Kick channel ${channelKickID} (chatroom ${chatroomId})`,
       );

-        console.log(
+        if (import.meta.env.VITE_DEBUG_7TV === 'true' || (typeof process !== 'undefined' && process.env.NODE_ENV === 'development')) console.debug(
           `[Shared7TV]: Received ${type} for ${chatroomId === null ? 'broadcast' : `chatroom ${chatroomId}`}`,
           {
             channelContext: body?.context || body?.condition || null,
             entitlementUser: body?.object?.user?.username,
           },
         );

-            console.log(
+            if (import.meta.env.VITE_DEBUG_7TV === 'true' || (typeof process !== 'undefined' && process.env.NODE_ENV === 'development')) console.debug(
               `[Shared7TV]: Forwarding cosmetic catalog update to ${chatroomId === null ? 'all chatrooms' : chatroomId}`,
               {
                 badgeCount: cosmetics?.badges?.length,
                 paintCount: cosmetics?.paints?.length,
               },
             );

-              console.log(
+              if (import.meta.env.VITE_DEBUG_7TV === 'true' || (typeof process !== 'undefined' && process.env.NODE_ENV === 'development')) console.debug(
                 `[Shared7TV]: Forwarding entitlement for ${body?.object?.user?.username || 'unknown user'}`,
                 {
                   chatroomId,
                   badgeId: body?.object?.user?.style?.badge_id,
                   paintId: body?.object?.user?.style?.paint_id,
                 },
               );

-      console.log(
+      if (import.meta.env.VITE_DEBUG_7TV === 'true' || (typeof process !== 'undefined' && process.env.NODE_ENV === 'development')) console.debug(
         `[Shared7TV]: Unable to directly map ${type} to a chatroom, broadcasting`,
         {
           contextId,
           knownChatrooms: Array.from(this.chatrooms.values()).map((data) => data.channelKickID),
         },
       );

Also applies to: 405-407, 485-487, 516-518, 592-599, 628-634, 649-656, 696-704

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

src/renderer/src/providers/ChatProvider.jsx (3)

1514-1525: Gate high-volume 7TV event logs behind a debug switch.

These logs run for every WS event and can flood consoles; also they include user identifiers. Wrap with a debug guard like the presence logs.

-          console.log(`[ChatProvider] Received 7TV event from shared WebSocket`, {
+          if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(`[ChatProvider] Received 7TV event from shared WebSocket`, {
             type,
             chatroomId,
             hasBody: !!body,
             entitlementUser: body?.object?.user?.username,
             badgeId: body?.object?.user?.style?.badge_id,
             paintId: body?.object?.user?.style?.paint_id,
             badgeCount: body?.badges?.length,
             paintCount: body?.paints?.length
-          });
+          });
-            console.log(`[ChatProvider] Routing event ${type} to specific chatroom: ${chatroomId}`);
+            if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(`[ChatProvider] Routing event ${type} to specific chatroom: ${chatroomId}`);
-            console.log(`[ChatProvider] Broadcasting event ${type} to all chatrooms (${chatrooms.length} total)`);
+            if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(`[ChatProvider] Broadcasting event ${type} to all chatrooms (${chatrooms.length} total)`);

Also applies to: 1527-1527, 1531-1531

---

1913-1920: Gate diagnostic per-event log or make it debug-only.

Same rationale as above; large volume and includes identifiers.

-    console.log(
+    if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(
       `[ChatProvider] Received 7TV event ${type} for ${chatroomId ?? 'broadcast'}`,
       {
         badgeCount: body?.badges?.length,
         paintCount: body?.paints?.length,
         entitlementUser: body?.object?.user?.username,
       },
     );

---

4092-4094: Minor: align presence interval with comment or update comment.

The comment says “every 2 minutes” but the interval is 1 minute. Pick one.

-      1 * 60 * 1000,
+      2 * 60 * 1000,

utils/services/seventv/sharedStvWebSocket.js (4)

346-361: Gate subscription/setup logs or sample them.

These run on every (re)connect; make them conditional to avoid noise.

-      console.log(`[Shared7TV]: Starting subscription to all events for ${this.chatrooms.size} chatrooms`);
+      if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(`[Shared7TV]: Starting subscription to all events for ${this.chatrooms.size} chatrooms`);
...
-      console.log(`[Shared7TV]: Finished subscribing to all events`);
+      if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(`[Shared7TV]: Finished subscribing to all events`);
...
-      console.log(`[Shared7TV]: Setting up message handler`);
+      if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(`[Shared7TV]: Setting up message handler`);
...
-      console.log(`[Shared7TV]: Message handler setup complete`);
+      if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(`[Shared7TV]: Message handler setup complete`);

Also applies to: 363-366

---

619-630: Reduce per-message logging; add debug guard and/or sampling.

Logging raw payloads and every opcode will spam consoles and degrade perf.

-    this.chat.onmessage = (event) => {
-      console.log(`[Shared7TV]: Raw WebSocket message received:`, event.data);
+    this.chat.onmessage = (event) => {
+      const __dbg = !!window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.();
+      if (__dbg) console.log(`[Shared7TV]: Raw WebSocket message received`);
       try {
         const msg = JSON.parse(event.data);
-        console.log(`[Shared7TV]: Parsed message:`, {
+        if (__dbg) console.log(`[Shared7TV]: Parsed message:`, {
           op: msg?.op,
           type: msg?.d?.type,
           hasBody: !!msg?.d?.body,
-          fullMessage: msg
+          // Avoid dumping full message outside deep debug
+          // fullMessage: msg
         });

Optionally, sample with a counter (e.g., log every Nth heartbeat/dispatch).

Also applies to: 632-743

---

753-766: Consider redacting or gating full cosmetic/entitlement bodies.

Full bodies include user identifiers; prefer debug-only or field-level logs already present.

---

888-896: Map cosmetic/entitlement events to chatrooms when context.id matches.

You always broadcast; use context.id to route when possible.

-    if (type.startsWith("cosmetic.") || type.startsWith("entitlement.")) {
-      const contextId = body?.context?.id || body?.condition?.id || null;
-      console.log(
-        `[Shared7TV]: Unable to directly map ${type} to a chatroom, broadcasting`,
-        {
-          contextId,
-          knownChatrooms: Array.from(this.chatrooms.values()).map((data) => data.channelKickID),
-        },
-      );
-    }
+    if (type.startsWith("cosmetic.") || type.startsWith("entitlement.")) {
+      const contextId = body?.context?.id || body?.condition?.id;
+      if (contextId != null) {
+        for (const [id, data] of this.chatrooms) {
+          if (String(data.channelKickID) === String(contextId)) {
+            return id;
+          }
+        }
+      }
+      if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) {
+        console.log(`[Shared7TV]: Broadcasting ${type} (no chatroom match)`, {
+          contextId,
+          knownChatrooms: Array.from(this.chatrooms.values()).map((d) => d.channelKickID),
+        });
+      }
+    }

utils/services/seventv/stvWebsocket.js (1)

355-466: Add a debug guard for per-opcode logs.

Mirrors the shared WS suggestion; these fire frequently.

-        switch (msg?.op) {
+        const __dbg = !!window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.();
+        switch (msg?.op) {
           case 0:
-            console.log(`[7TV]: Dispatch event received for channel ${this.channelKickID}`, { type: msg?.d?.type });
+            if (__dbg) console.log(`[7TV]: Dispatch event received for channel ${this.channelKickID}`, { type: msg?.d?.type });
             try {
               window.app?.telemetry?.recordWebSocketMessage?.(`7tv_${this.channelKickID}`, 'dispatch', msg?.d?.type);
             } catch (_) {}
             break;
          ...

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

src/renderer/src/providers/ChatProvider.jsx (3)

1514-1531: Gate verbose 7TV routing logs behind debug flag.

These logs fire for every event and chatroom; wrap them with shouldLogDebug to avoid hot‑path noise.

Apply:

-          console.log(`[ChatProvider] Received 7TV event from shared WebSocket`, {
+          if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(`[ChatProvider] Received 7TV event from shared WebSocket`, {
             type,
             chatroomId,
             hasBody: !!body,
             entitlementUser: body?.object?.user?.username,
             badgeId: body?.object?.user?.style?.badge_id,
             paintId: body?.object?.user?.style?.paint_id,
             badgeCount: body?.badges?.length,
             paintCount: body?.paints?.length
-          });
+          });
...
-            console.log(`[ChatProvider] Routing event ${type} to specific chatroom: ${chatroomId}`);
+            if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(`[ChatProvider] Routing event ${type} to specific chatroom: ${chatroomId}`);
...
-            console.log(`[ChatProvider] Broadcasting event ${type} to all chatrooms (${chatrooms.length} total)`);
+            if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(`[ChatProvider] Broadcasting event ${type} to all chatrooms (${chatrooms.length} total)`);

---

1913-1921: Gate per‑event 7TV logs.

Same rationale: avoid hot‑path logging unless debug is enabled.

Apply:

-    console.log(
+    if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) console.log(
       `[ChatProvider] Received 7TV event ${type} for ${chatroomId ?? 'broadcast'}`,
       {
         badgeCount: body?.badges?.length,
         paintCount: body?.paints?.length,
         entitlementUser: body?.object?.user?.username,
       },
     );

---

4087-4102: Presence interval comment mismatches code.

Comment says “every 2 minutes,” code runs every 1 minute. Align one of them.

Apply one (doc-only) fix:

-    // Send presence updates every 2 minutes
+    // Send presence updates every 1 minute

Or change interval to 2 * 60 * 1000.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

utils/services/connectionManager.js (1)

121-136: Gate Stv handler registration logs behind telemetry debug; downgrade missing-handler warning.

These logs will fire for every chatroom and can get noisy. Use shouldLogDebug and prefer console.debug for the “missing handler” case.

Apply this diff and add the helper shown below:

-      console.log(`[ConnectionManager] Registering onStvMessage handler`);
+      if (shouldLogDebug()) console.log(`[ConnectionManager] Registering onStvMessage handler`);
       this.stvWebSocket.addEventListener("message", handlers.onStvMessage);
-    } else {
-      console.warn(`[ConnectionManager] No onStvMessage handler provided!`);
+    } else {
+      if (shouldLogDebug()) console.debug(`[ConnectionManager] No onStvMessage handler provided`);
     }
@@
-      console.log(`[ConnectionManager] Registering onStvOpen handler`);
+      if (shouldLogDebug()) console.log(`[ConnectionManager] Registering onStvOpen handler`);
@@
-      console.log(`[ConnectionManager] Registering onStvConnection handler`);
+      if (shouldLogDebug()) console.log(`[ConnectionManager] Registering onStvConnection handler`);

Add near the top of the module:

// At top-level (once)
const shouldLogDebug = globalThis?.window?.__KT_TELEMETRY_UTILS__?.shouldLogDebug || (() => false);

src/renderer/src/telemetry/webTracing.js (5)

198-224: Renderer debug helpers OK; apply them consistently to top-level logs.

There are many unconditional console.logs earlier (e.g., Lines 32–35, 41, 49–50, 87–88, 120, 176–183, 189–196). Gate them with shouldLogDebug() for parity.

Example:

-console.log('[DEBUG] webTracing.js module loading - telemetry enabled:', telemetryEnabled);
+if (shouldLogDebug()) console.log('[DEBUG] webTracing.js module loading - telemetry enabled:', telemetryEnabled);

---

676-689: Exporter debug logs gated correctly. Consider redacting or truncating identifiers.

Not critical, but you can slice traceIds/spanIds to first 8–16 chars for compact logs.

Also applies to: 693-702, 709-720

---

1083-1111: IPCSimpleSpanProcessor still logs noisily; gate with shouldLogDebug().

These logs will emit for every span and can flood consoles.

-          console.log('[Renderer OTEL] Span started:', {
+          if (shouldLogDebug()) console.log('[Renderer OTEL] Span started:', {
@@
-            console.log('[Renderer OTEL] Span ended → exporting via IPC:', {
+            if (shouldLogDebug()) console.log('[Renderer OTEL] Span ended → exporting via IPC:', {
@@
-              console.log('[Renderer OTEL] Span export callback:', {
+              if (shouldLogDebug()) console.log('[Renderer OTEL] Span export callback:', {

Also applies to: 1113-1121, 1127-1135

---

1414-1601: Dev helpers (emitRendererTestSpan, compareOTLPPayloads) should be dev-only or gated.*

Expose these only in DEV or when shouldLogDebug() to avoid accidental use in production.

-// Expose helper to emit a manual renderer test span from DevTools
-if (true) {
+// Expose helpers only in dev or verbose debug
+if (import.meta.env.DEV || shouldLogDebug()) {
   // ... keep helper definitions ...
 }

---

2054-2055: Provider name mismatch breaks forceFlush in verifyGrafanaTraces.

You set window.KT_OTEL_PROVIDER earlier; here you read KT_TRACE_PROVIDER.

-        const provider = window.__KT_TRACE_PROVIDER__;
+        const provider = window.__KT_OTEL_PROVIDER__;

src/main/index.js (1)

715-719: OTLP IPC relay logs gated properly; add payload size guard to protect main thread.

Large payloads can stall the event loop. Reject overly large bodies early.

     const body = Buffer.from(JSON.stringify(exportJson), "utf8");
+    // Guardrail: reject payloads >5MB
+    const MAX_BYTES = 5 * 1024 * 1024;
+    if (body.length > MAX_BYTES) {
+      console.warn(`[OTEL IPC Relay][${requestId}] Rejecting oversized payload: ${body.length} bytes`);
+      return { ok: false, reason: "payload_too_large", requestId };
+    }

Also applies to: 764-767, 776-779, 799-801

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: CR
PR: BP602/KickTalk#0
File: AGENTS.md:0-0
Timestamp: 2025-09-18T22:47:22.731Z
Learning: Applies to src/{telemetry,renderer/src/telemetry}/**/*.{ts,tsx,js,jsx} : Configure OTLP HTTP export to Grafana Cloud; read endpoints/keys from env with main-safe prefixes; do not expose secrets to renderer

Learnt from: CR
PR: BP602/KickTalk#0
File: AGENTS.md:0-0
Timestamp: 2025-09-18T22:47:22.731Z
Learning: Applies to src/{telemetry,renderer/src/telemetry}/**/*.{ts,tsx,js,jsx} : Place OpenTelemetry instrumentation under src/telemetry/** and src/renderer/src/telemetry/**

src/renderer/src/components/Messages/ModActionMessage.jsx (1)

15-44: Missing dependency in useCallback.

userChatroomInfo is referenced but not in deps; could capture stale value.

-  );
+  );
   // ...
-    [chatroomName, username, chatroomId, allStvEmotes, subscriberBadges],
+    [chatroomName, username, chatroomId, allStvEmotes, subscriberBadges, userChatroomInfo],

src/renderer/src/components/Messages/Message.jsx (1)

44-55: Avoid high-frequency getUserStyle calls (noise + cost).

This selector calls a noisy getter every render; it warns if called <100ms apart. Derive from state directly to avoid logs, or gate logs behind a debug flag.

-  const subscribedUserStyle = useCosmeticsStore(
-    useShallow((state) => {
-      if (!message?.sender || type === "replyThread" || type === "dialog") {
-        return null;
-      }
-      return state.getUserStyle(message.sender.username);
-    }),
-  );
+  const subscribedUserStyle = useCosmeticsStore(useShallow((state) => {
+    if (!message?.sender || type === "replyThread" || type === "dialog") return null;
+    const uname = message.sender.username?.toLowerCase()?.replaceAll('-', '_');
+    const us = state.userStyles[uname];
+    if (!us?.badgeId && !us?.paintId) return null;
+    const badge = state.globalCosmetics.badges?.find((b) => b.id === us.badgeId);
+    const paint = state.globalCosmetics.paints?.find((p) => p.id === us.paintId);
+    return { badge, paint, color: us.color, username: us.username };
+  }));

utils/services/seventv/stvAPI.js (2)

254-356: Function name and unused param.

This fetches a global catalog and ignores channelId. Either rename (e.g., getCosmeticsCatalog) or document and remove the param.

Would you prefer we rename the function and update preload/ChatProvider accordingly?

---

260-265: Add request timeout and basic retry.

Network call should have a timeout to avoid hanging.

-    const response = await axios.get(`https://api.7tv.app/v2/cosmetics`);
+    const response = await axios.get(`https://api.7tv.app/v2/cosmetics`, { timeout: 10000 });

src/renderer/src/providers/CosmeticsProvider.jsx (2)

61-63: Connection discriminator mismatch ('type' vs 'platform').

Elsewhere you check c.platform === "KICK". Use the same key here.

-            kickConnection: userStyle.connections?.find((c) => c.type === "KICK"),
+            kickConnection: userStyle.connections?.find((c) => c.platform === "KICK"),

---

11-21: Gate verbose diagnostics behind a debug flag.

Logs are very chatty and executed in hot paths; wrap with a feature flag (e.g., window.KT_TELEMETRY_UTILS?.shouldLogDebug?.()).

Also applies to: 22-31, 35-45, 51-53, 170-179, 188-196, 205-213, 215-220, 228-234, 257-279, 305-308

src/renderer/src/providers/ChatProvider.jsx (1)

555-557: Gate debug logs.

Use your telemetry debug flag for these presence logs to keep consoles clean.

-      if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) {
+      if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) {
         console.log("[7tv Presence]: Last update time for chatroom:", userId, lastUpdateTime, stvPresenceUpdates);
       }

-          if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) {
+          if (window.__KT_TELEMETRY_UTILS__?.shouldLogDebug?.()) {
             console.log("[7tv Presence]: Sending presence check for chatroom:", chatroom.streamerData.user_id);
           }

Also applies to: 4131-4133

utils/services/seventv/sharedStvWebSocket.js (7)

213-218: Also (re)subscribe user. when adding a chatroom on an already-open socket*

Without this, user events for newly added chatrooms won't be subscribed.

   if (this.connectionState === 'connected') {
     console.log(`[Shared7TV DIAGNOSTIC]: Subscribing to events for new chatroom ${chatroomId}`);
     this.subscribeToChatroomEvents(chatroomId);
+    // ensure user-scoped subscriptions are up to date
+    this.subscribeToUserEvents();
   } else {

---

31-37: Badge URL construction lacks host/files guards; can throw if host/files missing

Make it resilient to missing host or empty files array.

-      const newBadge = {
-        id: data.id === "00000000000000000000000000" ? data.ref_id || "default_id" : data.id,
-        title: data.tooltip,
-        url: `https:${data.host.url}/${data.host.files[data.host.files.length - 1].name}`,
-      };
+      const id = data.id === "00000000000000000000000000" ? (data.ref_id || "default_id") : data.id;
+      const baseUrl = typeof data?.host?.url === "string"
+        ? (data.host.url.startsWith("//") ? `https:${data.host.url}` : data.host.url)
+        : null;
+      const lastFile = Array.isArray(data?.host?.files) && data.host.files.length
+        ? data.host.files[data.host.files.length - 1].name
+        : null;
+      const newBadge = {
+        id,
+        title: data.tooltip,
+        url: baseUrl && lastFile ? `${baseUrl}/${lastFile}` : null,
+      };

---

824-837: Check entitlement body shape before filtering by kind

Some 7TV entitlement events carry kind under body.object.kind; guard both to avoid dropping valid events.

-        // Process entitlements for both cosmetics (kind 10) and emote sets (kind 5)
-        if (body.kind === 10 || body.kind === 5) {
+        // Process entitlements for both cosmetics (kind 10) and emote sets (kind 5)
+        const kind = body?.kind ?? body?.object?.kind;
+        if (kind === 10 || kind === 5) {

---

381-395: Gate verbose console logs behind a debug flag to reduce prod noise

Keep diagnostics, but guard with a single isDebug flag or telemetry-only in production.

- console.log(`[Shared7TV]: ...`);
+ if (window.__DEBUG_FLAGS__?.stv) console.log(`[Shared7TV]: ...`);

Also applies to: 397-400, 465-498, 542-569, 571-598, 633-757, 121-123, 37-37, 215-218

---

767-771: Avoid logging PII (usernames) unless gated

Usernames can be considered PII; ensure debug-only.

Also applies to: 775-782

---

173-204: Store STV user ID variant for routing (consistency with idVariants block)

Add stvId into idVariants to improve mapping.

   if (chatroomData) {
     storedData.idVariants = {
       chatroom_id: String(chatroomId),
       streamer_id: String(channelKickID),
       streamer_user_id: chatroomData.streamerData?.user_id ? String(chatroomData.streamerData.user_id) : null,
       username: chatroomData.streamerData?.username || null,
       user_username: chatroomData.streamerData?.user?.username || null,
       slug: chatroomData.slug || null,
+      stv_user_id: storedData.stvId || null,
     };

---

865-872: Emote set routing: body.id might be nested; add fallback to body.object.id/object_id

Prevents misrouting when the event schema differs.

-    if (type.startsWith("emote_set.") && body?.id) {
+    if (type.startsWith("emote_set.") && (body?.id || body?.object_id || body?.object?.id)) {
       for (const [chatroomId, data] of this.chatrooms) {
-        if (data.stvEmoteSetId === body.id) {
+        const incomingId = body?.id ?? body?.object_id ?? body?.object?.id;
+        if (data.stvEmoteSetId === incomingId) {
           return chatroomId;
         }
       }
     }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: CR
PR: BP602/KickTalk#0
File: AGENTS.md:0-0
Timestamp: 2025-09-18T22:47:22.731Z
Learning: Applies to src/preload/**/*.{ts,js,mts,mjs,cjs} : Expose a minimal, purpose-built API from preload via contextBridge.exposeInMainWorld